Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Audit 380 guns #42718

Merged
merged 4 commits into from
Aug 4, 2020
Merged

Audit 380 guns #42718

merged 4 commits into from
Aug 4, 2020

Conversation

feinorgh
Copy link
Contributor

@feinorgh feinorgh commented Aug 4, 2020

Summary

SUMMARY: Content "Audit .380 caliber guns for volume, weight, and longest_side"

Purpose of change

Contribute to #37374 and #41691

Describe the solution

Update the JSON file the calculated values, from the method described in #37374

Describe alternatives you've considered

Cocktails, cocktails, cocktails.

Testing

Test suite passes. JSON validates. Guns spawn in-game with proper values.

Additional context

This should conclude the .380 caliber guns that were missing "longest_side".

Measurements are recorded here: https://docs.google.com/spreadsheets/d/10TeuTTjwusbWB9SaH-nq03bMWDyILylU9Btvtu3uTDM/edit#gid=835787504&range=A2

FN 1910

There was no reliable source for the width of the weapon. Eventually, I found a top view image, edited away the stuff around it and measured the width of the slide. This is the value of 17.6 mm that went into width. Someone else might want to double check this, even though the weapon looks thin in the images.

111 - FN 1910  380

https://docs.google.com/spreadsheets/d/10TeuTTjwusbWB9SaH-nq03bMWDyILylU9Btvtu3uTDM/edit#gid=93328453&range=A46

Kel-Tec P3AT

112 - Kel-Tec P3AT

https://docs.google.com/spreadsheets/d/10TeuTTjwusbWB9SaH-nq03bMWDyILylU9Btvtu3uTDM/edit#gid=93328453&range=A78

Hi-Point CF-380

113 - Hi-Point CF-380

https://docs.google.com/spreadsheets/d/10TeuTTjwusbWB9SaH-nq03bMWDyILylU9Btvtu3uTDM/edit#gid=93328453&range=A70

Taurus Spectrum

114 - Taurus Spectrum

https://docs.google.com/spreadsheets/d/10TeuTTjwusbWB9SaH-nq03bMWDyILylU9Btvtu3uTDM/edit#gid=93328453&range=A186

@ZhilkinSerg ZhilkinSerg added [JSON] Changes (can be) made in JSON Items / Item Actions / Item Qualities Items and how they work and interact labels Aug 4, 2020
@ZhilkinSerg ZhilkinSerg merged commit 515bf04 into CleverRaven:master Aug 4, 2020
@feinorgh feinorgh deleted the audit_380_guns branch August 6, 2020 17:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Items / Item Actions / Item Qualities Items and how they work and interact [JSON] Changes (can be) made in JSON
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants